Skip to content

Conversation

@guardrex
Copy link
Collaborator

@guardrex guardrex commented Jan 7, 2025

Fixes #34229

Javier ... If u see this later and have changes, you can message me here or offline for additional updates. I think this is a an improvement in this form, and I'd like to get it in before taking off for the weekend.

Thanks @MattCost! 🚀 ... I like the direction that this is headed.

UPDATE (1/8): I slept on it, and I didn't like what I placed yesterday. I like the latest commit the best.

  • I added cross-links from the NOTEs to the relevant (existing) section that Mackinnon and I placed earlier.
  • I enhanced the existing section that Mackinnon and I placed earlier, bringing in some of Javier's sentiment/remarks.

Also as part of this, I reviewed the content in the other locations that I mentioned on the issue, and I think that content is fine. No changes in those spots at this time.


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/components/prerender.md aspnetcore/blazor/components/prerender

@guardrex guardrex self-assigned this Jan 7, 2025
@guardrex guardrex requested a review from javiercn January 8, 2025 15:08
@guardrex guardrex merged commit 726ca7c into main Jan 10, 2025
3 checks passed
@guardrex guardrex deleted the guardrex/blazor-enhanced-nav-prerender-persist-state branch January 10, 2025 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Blazor Pre-rendering and PersistingComponentStateSubscription not working correctly with dotnet 9?

2 participants